Skip to content

Integration - #506

Merged
al1img merged 5 commits into
aosedge:feature_unificationfrom
al1img:integration
Feb 10, 2026
Merged

Integration#506
al1img merged 5 commits into
aosedge:feature_unificationfrom
al1img:integration

Conversation

@al1img

@al1img al1img commented Feb 9, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings February 9, 2026 19:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@MykolaSuperman MykolaSuperman left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Mykola Solianko mykola_solianko@epam.com

@mykola-kobets-epam mykola-kobets-epam left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Mykola Kobets <mykola_kobets@epam.com>

@al1img
al1img force-pushed the integration branch 2 times, most recently from 216c8f9 to 4c231cb Compare February 10, 2026 10:57
@codecov

codecov Bot commented Feb 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.03540% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.22%. Comparing base (3bfeb08) to head (1505380).
⚠️ Report is 5 commits behind head on feature_unification.

Files with missing lines Patch % Lines
src/core/cm/updatemanager/unitstatushandler.cpp 86.00% 7 Missing ⚠️
src/core/cm/launcher/launcher.cpp 78.26% 5 Missing ⚠️
src/core/cm/imagemanager/imagemanager.cpp 86.66% 2 Missing ⚠️
src/core/cm/launcher/node.cpp 84.61% 2 Missing ⚠️
src/core/cm/updatemanager/desiredstatushandler.cpp 92.00% 2 Missing ⚠️
Additional details and impacted files
@@                   Coverage Diff                   @@
##           feature_unification     #506      +/-   ##
=======================================================
+ Coverage                85.18%   85.22%   +0.03%     
=======================================================
  Files                      308      308              
  Lines                    27186    27330     +144     
  Branches                  3664     3684      +20     
=======================================================
+ Hits                     23158    23291     +133     
- Misses                    4028     4039      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Oleksandr Grytsov <oleksandr_grytsov@epam.com>
Reviewed-by: Mykola Kobets <mykola_kobets@epam.com>
Reviewed-by: Mykola Solianko <mykola_solianko@epam.com>
Signed-off-by: Oleksandr Grytsov <oleksandr_grytsov@epam.com>
Reviewed-by: Mykola Kobets <mykola_kobets@epam.com>
Reviewed-by: Mykola Solianko <mykola_solianko@epam.com>
Unit config and node handlers don't provide updating statuses. Store and handle
them locally in unit status handler.

Signed-off-by: Oleksandr Grytsov <oleksandr_grytsov@epam.com>
Reviewed-by: Mykola Kobets <mykola_kobets@epam.com>
Reviewed-by: Mykola Solianko <mykola_solianko@epam.com>
Signed-off-by: Oleksandr Grytsov <oleksandr_grytsov@epam.com>
Reviewed-by: Mykola Kobets <mykola_kobets@epam.com>
Reviewed-by: Mykola Solianko <mykola_solianko@epam.com>
Use info logs for important for update items and instances events.

Signed-off-by: Oleksandr Grytsov <oleksandr_grytsov@epam.com>
Reviewed-by: Mykola Kobets <mykola_kobets@epam.com>
Reviewed-by: Mykola Solianko <mykola_solianko@epam.com>
@sonarqubecloud

Copy link
Copy Markdown

@al1img
al1img merged commit 6f2cc7b into aosedge:feature_unification Feb 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants